Audit identified 55+ clippy warnings from redundant code patterns. All functional issues resolved.

Changes

Redundant closures (10 instances)

// Before
.map(|s| simplify_model_name(s))

// After
.map(simplify_model_name)

Dead conditional branches (9 instances)

// Before
pub fn api_key_display_label() -> &'static str {
    if is_chinese() {
        "API Key"
    } else {
        "API Key"
    }
}

// After
pub fn api_key_display_label() -> &'static str {
    "API Key"
}

API improvements

• Changed &PathBuf to &Path in function signatures
• Removed unnecessary reference in extract_api_url call

Test fixes

• Migrated async tests from unavailable tauri::async_runtime to #[tokio::test]

Notes

33 dead code warnings remain - these are unused library API functions. One integration test (deeplink_import) fails due to unimplemented feature.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.